Skip to content

fix (color scheme): add blocksy color scheme compatibility#3623

Closed
kaeizen wants to merge 1 commit intodevelopfrom
fix/blocksy-color-scheme-compatibility
Closed

fix (color scheme): add blocksy color scheme compatibility#3623
kaeizen wants to merge 1 commit intodevelopfrom
fix/blocksy-color-scheme-compatibility

Conversation

@kaeizen
Copy link
Contributor

@kaeizen kaeizen commented Oct 9, 2025 •
edited by coderabbitai bot
Loading

Summary by CodeRabbit

  • New Features
    • Added compatibility with the Blocksy theme’s color schemes. Headings and body text now automatically map to the theme’s heading and text colors, delivering consistent styling across the site when Blocksy color schemes are active. This ensures better legibility and alignment with your chosen color palette without additional configuration, and applies automatically when the Blocksy theme is detected.

@coderabbitai
Copy link

coderabbitai bot commented Oct 9, 2025 •
edited
Loading

Walkthrough

Adds Blocksy theme compatibility styles by defining CSS custom properties for heading and text colors under the selector .stk--is-blocksy-theme.stk-has-color-schemes in src/compatibility/blocksy/style.scss.

Changes

Cohort / File(s) Summary
Blocksy theme color variables
src/compatibility/blocksy/style.scss		 Introduces CSS variables for Blocksy color schemes: sets --stk-heading-color via a fallback chain to Blocksy heading vars and maps --stk-text-color to --theme-text-color.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Poem

A nibble of CSS in Blocksy’s glade,
I set the hues where headings are made.
Text gets its tone, a gentle delight,
Variables whisper, colors align right.
Thump goes my foot—deploy tonight! 🐇🎨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title clearly and concisely describes the primary change by specifying that Blocksy color scheme compatibility is being added as a fix to the color scheme functionality, matching the changeset’s focus on theming adjustments. It avoids unnecessary detail while remaining specific to the main update.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/blocksy-color-scheme-compatibility
📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 578783c and c1d7409.

📒 Files selected for processing (1)
  • src/compatibility/blocksy/style.scss (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (7)
  • GitHub Check: PHP 8.2 and WP latest
  • GitHub Check: PHP 7.3 and WP 6.5.5
  • GitHub Check: PHP 8.2 and WP 6.7.2
  • GitHub Check: PHP 7.3 and WP latest
  • GitHub Check: PHP 8.2 and WP 6.5.5
  • GitHub Check: PHP 8.2 and WP 6.6.2
  • GitHub Check: build
🔇 Additional comments (1)
src/compatibility/blocksy/style.scss (1)

1-4: CSS variable mapping looks solid

Using :where keeps specificity flat while wiring Stackable heading/text tokens into Blocksy’s color scheme variables—fallback chain reads correctly. Nice drop-in compatibility tweak.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@kaeizen kaeizen self-assigned this Oct 9, 2025
@github-actions
Copy link

github-actions bot commented Oct 9, 2025

🤖 Pull request artifacts

file commit
pr3623-stackable-3623-merge.zip c1d7409

github-actions bot added a commit that referenced this pull request Oct 9, 2025
@github-actions
Upload pr3623-stackable-3623-merge.zip (c1d74)
fd987c7 
Pull request: #3623
Commit: c1d7409
@github-actions
Copy link

github-actions bot commented Oct 9, 2025

Try this Pull Request in the WordPress playground: https://playground.wordpress.net/?mode=seamless#{"landingPage":"/wp-admin/post-new.php?post_type=page","preferredVersions":{"php":"latest","wp":"latest"},"steps":[{"step":"login","username":"admin","password":"password"},{"step":"installPlugin","pluginZipFile":{"resource":"url","url":"https://raw.githubusercontent.com/gambitph/Stackable/artifacts/pr3623-stackable-3623-merge.zip"}}]}

@kaeizen kaeizen closed this Oct 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@kaeizen kaeizen

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@kaeizen